Skip to content

fix(devops-copilot): Remove the mode toggle when the user has access to one mode only#2357

Merged
rmnbrd merged 2 commits intostagingfrom
fix/QOV-1558/remove-mode-toggle
Feb 4, 2026
Merged

fix(devops-copilot): Remove the mode toggle when the user has access to one mode only#2357
rmnbrd merged 2 commits intostagingfrom
fix/QOV-1558/remove-mode-toggle

Conversation

@killiancolla
Copy link
Collaborator

@killiancolla killiancolla commented Feb 2, 2026

Summary

Issue: QOV-1558

Disabled the mode toggle when the user does not have read-write access to prevent the toggle from being locked without knowing why

Screenshots / Recordings

Before After
image image

Testing

  • Changes tested locally in the relevant Console's pages and Storybooks
  • yarn test or yarn test -u (if you need to regenerate snapshots)
  • yarn format
  • yarn lint

PR Checklist

  • I followed naming, styling, and TypeScript rules (see .cursor/rules)
  • I performed a self-review (diff inspected, dead code removed)
  • I titled the PR using Conventional Commits with a scope when possible (e.g. feat(service): add new Terraform service) - required for semantic-release
  • I only kept necessary comments, written in English (watch for useless AI comments)
  • I involved a designer to validate UI changes if I am not a designer
  • I covered new business logic with tests (unit)
  • I confirmed CI is green (Codecov red can be accepted)
  • I reviewed and executed locally any AI-assisted code

@killiancolla killiancolla requested a review from rmnbrd February 2, 2026 12:49
@RemiBonnet
Copy link
Member

RemiBonnet commented Feb 2, 2026

Qovery Preview

Qovery can create a Preview Environment for this PR.
To trigger its creation, please post a comment with one of the following command.

Command Blueprint environment
/qovery preview 28c47145-c8e7-4b9d-8d9e-c65c95b48425 staging
/qovery preview cc1de7de-94e5-40f8-8e45-bc8986d9dfec storybook
/qovery preview {all|UUID1,UUID2,...} To preview multiple environments

This comment has been generated from Qovery AI 🤖.
Below, a word from its wisdom :

Make it work, make it right, make it fast

@codecov
Copy link

codecov bot commented Feb 2, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 50.52%. Comparing base (8219801) to head (5bf1556).
⚠️ Report is 2 commits behind head on staging.

Files with missing lines Patch % Lines
...configuration/section-ai-copilot-configuration.tsx 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files 
@@             Coverage Diff             @@
##           staging    #2357      +/-   ##
===========================================
- Coverage    52.38%   50.52%   -1.87%     
===========================================
  Files          381      305      -76     
  Lines         7949     6243    -1706     
  Branches      2432     1832     -600     
===========================================
- Hits          4164     3154    -1010     
+ Misses        3008     2514     -494     
+ Partials       777      575     -202
Flag Coverage Δ
unittests 50.52% <66.66%> (-1.87%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rmnbrd rmnbrd merged commit 3525e33 into staging Feb 4, 2026
9 of 10 checks passed
@rmnbrd rmnbrd deleted the fix/QOV-1558/remove-mode-toggle branch February 4, 2026 09:55
@RemiBonnet
Copy link
Member

RemiBonnet commented Feb 4, 2026

🎉 This PR is included in version 1.282.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rmnbrd rmnbrd rmnbrd approved these changes

Assignees

No one assigned

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@killiancolla @RemiBonnet @rmnbrd